From cc22469b6908e9e12ba77b02474aadd59bcaa990 Mon Sep 17 00:00:00 2001 From: Kristian Rietveld Date: Sun, 17 Oct 2010 10:27:41 +0200 Subject: [PATCH] Icon view must layout before the draw handler is called gtk_icon_view_layout(), which can resize the bin window and thus underlying surfaces, could be called from gtk_icon_view_draw(). So we have to make sure that layouting is done before the draw handler is called. We use the same construction as in tree view here, the layout idle handler is installed with a higher priority than GDK_PRIORITY_REDRAW and we wrap the call to gdk_window_process_updates() in a new function that will flush the layout queue beforehand. --- gtk/gtkiconview.c | 26 +++++++++++++++++++------- 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/gtk/gtkiconview.c b/gtk/gtkiconview.c index f8f830ae02..f151923c7f 100644 --- a/gtk/gtkiconview.c +++ b/gtk/gtkiconview.c @@ -58,6 +58,8 @@ #define SCROLL_EDGE_SIZE 15 +#define GTK_ICON_VIEW_PRIORITY_LAYOUT (GDK_PRIORITY_REDRAW + 5) + typedef struct _GtkIconViewItem GtkIconViewItem; struct _GtkIconViewItem { @@ -1564,11 +1566,6 @@ gtk_icon_view_draw (GtkWidget *widget, gtk_cairo_transform_to_window (cr, widget, icon_view->priv->bin_window); - /* If a layout has been scheduled, do it now so that all - * cell view items have valid sizes before we proceed. */ - if (icon_view->priv->layout_idle_id != 0) - gtk_icon_view_layout (icon_view); - cairo_set_line_width (cr, 1.); gtk_icon_view_get_drag_dest_item (icon_view, &path, &dest_pos); @@ -2659,6 +2656,19 @@ gtk_icon_view_real_toggle_cursor_item (GtkIconView *icon_view) } /* Internal functions */ +static void +gtk_icon_view_process_updates (GtkIconView *icon_view) +{ + /* Prior to drawing, we check if a layout has been scheduled. If so, + * do it now that all cell view items have valid sizes before we proceeed + * (and resize the bin_window if required). + */ + if (icon_view->priv->layout_idle_id != 0) + gtk_icon_view_layout (icon_view); + + gdk_window_process_updates (icon_view->priv->bin_window, TRUE); +} + static void gtk_icon_view_adjustment_changed (GtkAdjustment *adjustment, GtkIconView *icon_view) @@ -2672,7 +2682,7 @@ gtk_icon_view_adjustment_changed (GtkAdjustment *adjustment, if (icon_view->priv->doing_rubberband) gtk_icon_view_update_rubberband (GTK_WIDGET (icon_view)); - gdk_window_process_updates (icon_view->priv->bin_window, TRUE); + gtk_icon_view_process_updates (icon_view); } } @@ -3386,7 +3396,9 @@ gtk_icon_view_queue_layout (GtkIconView *icon_view) if (icon_view->priv->layout_idle_id != 0) return; - icon_view->priv->layout_idle_id = gdk_threads_add_idle (layout_callback, icon_view); + icon_view->priv->layout_idle_id = + gdk_threads_add_idle_full (GTK_ICON_VIEW_PRIORITY_LAYOUT, + layout_callback, icon_view, NULL); } static void -- 2.30.2